Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PROPOSED] Web Machine Learning Working Group Charter #19

Merged
merged 11 commits into from
Feb 2, 2023

Conversation

anssiko
Copy link
Member

@anssiko anssiko commented Oct 11, 2022

This is a work in progress draft charter for the Web Machine Learning Working Group. The charter builds upon the initial charter. Feel free to review the draft charter and raise issues.

The expected timeline is as follows:

  • Q4 '22: Charter development
  • Q1 '23: W3C Advisory Committee review
  • Q2' 23: Charter approved

See also: Diff to the initial charter.


Preview | Diff

@anssiko
Copy link
Member Author

anssiko commented Oct 11, 2022

I have opened issues to track feedback provided during the kick off call on 6 October 2022.

I will keep this draft PR updated to reflect the WG's consensus that is forming in the issue discussions.

Following this example, feel free to open more issues for feedback that has not yet been recorded. And please provide your comments in the existing open issues to help us prepare our next charter. Thank you for your contributions!

"base linear algebra operations are out of scope" refers to the
Basic Linear Algebra Subprograms (BLAS) interface.

Related #20
Confidence level increased.

Fix #21
These informative definitions will facilitate progressive lowering
by implementations.

Fix #22
Related #28
@anssiko
Copy link
Member Author

anssiko commented Dec 8, 2022

@anssiko anssiko marked this pull request as ready for review December 8, 2022 10:04
@anssiko anssiko requested a review from dontcallmedom December 8, 2022 10:05
Copy link
Member

@dontcallmedom dontcallmedom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • the Timeline section needs an update
  • The expected completion date of WebNN needs an update
  • have the prospects of the Model API integration in the charter evolved given the recent prototyping efforts?
  • the charter should probably link explicitly to the ethics doc now that it exists

Otherwise, I need to check if the charter template has evolved, but the proposed changes themselves look good to me.

- Update Expected completion date of WebNN
- Remove now redundant Timeline section
- Link to Ethical Principles for Web Machine Learning WG Note
- Add Adopted Draft and Exclusion Draft placeholders
- Update WebNN and Model Loader Draft state
@anssiko
Copy link
Member Author

anssiko commented Dec 9, 2022

@dontcallmedom thanks! I've pushed an update to address your review comments.

I also incorporated some updates from the evolved charter template, namely Adopted Draft and Exclusion Draft placeholders. I also observed the Timeline section is no longer included in some recent charters (notably GPU for the Web WG), so I dropped it.

have the prospects of the Model API integration in the charter evolved given the recent prototyping efforts?

The Model Loader API is still considered an incubation and its tentative specification status would allow it to advance should the conditions enumerated in the note be satisfied. I will loop in @jbingham to confirm the latest status of the prototype.

charter.html Outdated
the Coordination section for details.
To avoid overlap with existing work, alignment with the Basic
Linear Algebra Subprograms (BLAS) interface is out of scope. The
WebGL and WebGPU shaders and WebAssembly SIMD are expected to

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we already removed WebGL interop from the spec, should we also remove it here from the charter? Preferably noted as out of scope.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestions. I pushed 33e2eb5 that should address your comments. Namely:

I removed the WebGL reference from this BLAS compatibility context and added the following text following this paragraph:

Interoperability between the WebNN and WebGL APIs is out of scope.

With this change, the only other reference to WebGL in addition to the above out of scope statement remains in the Motivation and Background section as follows:

Currently, machine learning inference in the browser uses the WebGL graphics API with limited or no access to platform capabilities beneficial for ML such as CPU parallelism, general-purpose GPU, or dedicated ML hardware accelerators.

I think this is appropriate for context setting and to give a historical perspective. With WebNN API becoming more broadly available, the need to use WebGL API for ML workloads is expected to diminish over time.

@@ -237,11 +255,11 @@ <h3 id="out-of-scope">
algorithms.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We excluded training capability from the original charter. Do we want to rediscuss that aspect of the spec for webnn v2?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for raising this. I have opened #27 to solicit input on training capability. Please provide your feedback in that issue to feed into a broader WG discussion. We still have time to in-scope this if there is adequate interest.

- Remove reference to WebGL from the BLAS compatibility context
- Declare interop between the WebNN and WebGL APIs out of scope
Move examples of well-known model architectures from the bullet
list into the text section that talks about priority use cases.
Add transformers as another example.

Grammar fix: s/Allow/Allows

Fix #23
@dontcallmedom dontcallmedom merged commit 1fdb72b into main Feb 2, 2023
@anssiko
Copy link
Member Author

anssiko commented Feb 2, 2023

I met with @dontcallmedom and we merged the charter 2023-2025 PR. Dom is doing his final pass review before the charter starts its AC review.

@anssiko anssiko changed the title [DRAFT] Web Machine Learning Working Group Charter [PROPOSED] Web Machine Learning Working Group Charter Feb 14, 2023
@anssiko
Copy link
Member Author

anssiko commented Feb 14, 2023

(W3M & horizontal review in process: w3c/strategy#367)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants